Skip to content

Chore: [AEA-0000] - correct secret name in update dev container workflow#105

Merged
anthony-nhs merged 5 commits intomainfrom
fix_update_dev_container
Mar 27, 2026
Merged

Chore: [AEA-0000] - correct secret name in update dev container workflow#105
anthony-nhs merged 5 commits intomainfrom
fix_update_dev_container

Conversation

@anthony-nhs
Copy link
Copy Markdown
Contributor

Summary

  • Routine Change

Details

  • correct secret name in update dev container workflow

Copilot AI review requested due to automatic review settings March 25, 2026 16:20
@github-actions
Copy link
Copy Markdown
Contributor

This PR is linked to a ticket in an NHS Digital JIRA Project. Here's a handy link to the ticket:

AEA-0000

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the devcontainer version update workflow to use corrected GitHub App secret names so it can generate an app token and open/update the version bump PR.

Changes:

  • Renamed workflow_call secret inputs in update-dev-container-version.yml from AUTOMERGE_* to CREATE_PULL_REQUEST_*.
  • Updated the README usage example to pass the renamed secrets.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
README.md Updates the example snippet to use the new secret names.
.github/workflows/update-dev-container-version.yml Renames required workflow_call secrets and updates token creation step to reference them.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@anthony-nhs anthony-nhs enabled auto-merge (squash) March 26, 2026 10:28
@sonarqubecloud
Copy link
Copy Markdown

@anthony-nhs anthony-nhs merged commit 88f13d5 into main Mar 27, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants